window: Check if we can use CSD before enabling them

The change in 03213b9509fc1df16c66194ea168aed6c15110e9 changed the rules
as to when CSD can be enabled, but it also unconditionally enables CSD
with the implicit assumption that client-side shadows were the real
issue, and that we could work around that by drawing our own borders.
This also means that setting a titlebar for a GtkWindow will enable CSD
unconditionally.

In reality, some window managers (like Matchbox) *only* support
server-side decorations, and will ignore all hints to the contrary, to
the point of drawing decorations at random locations on top of the
window.

Since CSD are enabled unconditionally, the GTK_CSD environment variable
is also not a suitable escape hatch.

In the grand tradition of asking ourselves if we should do something
just because we can, we should split the environment checks from the
checks on what the user requested; by doing that, we can also check
when enabling client-side decorations, and ideally bail out if needed.

https://bugzilla.gnome.org/show_bug.cgi?id=750343

mir: keep reference on queued events

mir: gdk_mir_window_impl_set_surface_type

mir: Create the correct surface

mir: Remove redundant calls

mir: log input events properly

mir: Resolve warnings

mir: Update cursor position

mir: Remove child window rendering hack

wayland: Move DnD grab breaking function into gdkdevice-wayland.c

This has little to do with GdkDragContext, and a lot to do with
the GdkDevice that triggered it, seems to make more sense in
gdkdevice-wayland.c.

wayland: Split keyboard/button modifiers internally

On wl_keyboard.key/modifiers, we're just forgetting about currently
pressed mouse buttons. Fix this by storing button and key modifiers
separately, and put these together when creating the GdkEvents.

textview: Update handles after flushing scroll changes

Text insertion/pasting might trigger scroll, so we'd have to wait
until the text was revalidated and the scrolling truly happened
before we can check the new handle(s) position.

dnd: Remove emission of fake button release event

Nowadays (and for quite some years now) we rely on GdkEventGrabBroken
events to be received. This fake button event seems a remnant of
the early Gtk 2.x days, and is currently even inconsistent with our
event delivery model, so just remove it.

https://bugzilla.gnome.org/show_bug.cgi?id=749737

build: Fix make dist

Revert 2 hunks introduced by 36c65c1acabfd8fb1b5b85d7b128bfd4361b6fc3

configure: Require pango >= 1.37.0

Needed since 049cbc8b74aa7a6fff1627c8c65cbfd3c86dfde8

updateiconcache: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

queryimmodules: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

xembed: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

widget path: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

widget: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

tree view: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

tree model sort: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

tree model filter: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

tool palette: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

text buffer: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

print operation: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

paned: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

menu: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

kinetic scrolling: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

im context: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

icon view: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

icon theme: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

file chooser: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

entry: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

dnd: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

css: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

combo box: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

builder: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

application: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

accel map: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

about dialog: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

printing: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

x11: Fix compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

wayland: Avoid compiler warnings

Fix warnings due to -Wdeclaration-after-statement and -Wshadow.

Updated Tajik translation

Bug 745957 - GTK+/Quartz >= 3.14.8: Symbol not found: _gtk_drag_cancel

Implement gtk_drag_cancel, made public in c160ba069 and 62616a71, in
gtkdnd-quartz.c.

GtkStack: Don't mix code and declarations

Trivial: Use a convenience macro

g_strstrip is less obscure than the two individual functions,
so lets use it here.

file chooser: Respect recent settings

When recent files are not supported in gvfs, or turned off by
settings, we should not try to load them even if the startup
mode says so. This prevents inconsistency with the places
sidebar where 'Recent' will not appear in this case.

placessidebar: Simplify some code

Use existing functions, instead of open-coding manipulation
of string arrays.

flowbox: Fix compiler warnings

Introduced in e59e38b58120f8d998b5e204506fc552e2efd00d

cssnode: Fix position invalidation

The previous code was overly complicated and wrong.

GtkFlowBox: support positional css selectors

As in the previous commit for GtkListBox, support :first-child,
:last-child, :nth-child() and :last-nth-child() selectors here.

listbox: add support for :first-child, :last-child and others

This applies the proper CSS child ordering semantics using GtkCssNode to
GtkListBox. You can now use :first-child, :last-child, :nth-child(), and
:last-nth-child() selectors.

For example, this allows styling row separators using CSS while ignoring
the separator on the last row.

 GtkListBoxRow {
   border-bottom: 1px solid @borders;
 }
 GtkListBoxRow:last-child {
   border-bottom: none;
 }

When the sort ordering of the listbox changes, we also update the CSS
node ordering.

Updated Occitan translation

Fix DnD Introspection on Quartz Backend.

Don't exclude the -quartz.c files from introspection_files.

GtkStack: Interpolate between size changes

GtkStack: Remove some useless function calls

We will call gtk_widget_queue_resize anyway later when finishing the
(maybe non-running) transition, which will also cause the stack to be
repainted.

GtkStack: Keep the last_visible_child around during transitions

We need it later.

GtkStack: Misc indentation fixes

GtkStack: Remove useless cast

GtkStack: Remove useless function calls

These both just use the default values anyway.

GtkStack: Save last visible widget size

When interpolating the stack size, we compute the current size by using
the prefered/minimum current size and the last size. We can't use the
last_visible_surface_allocation because that is not available until the
first _draw call and it doesn't include the child's margins.

GtkStack: Remove gtk_widget_set_opacity call

The corresponding call with 0.999 opacity is gone, so we don't need to
reset the opacity anymore.

gtkpopover: Update the menu's alignment

... when the popover position changes.

Fix GDK MSVC Projects

We need to link to dwmapi.lib for all configs, not just the Broadway
ones, so fix the build.

Fix a cornercase crash

If gtk_text_layout_move_cursor_visually is called with a
count of 0, we were passing NULL to a free function that
can't handle it. Don't do that.

https://bugzilla.gnome.org/show_bug.cgi?id=750058

cssmatcher: Speed up common matching

first-child and last-child are the most common usages of the nth-child
machinery, so special-casing them makes sense.

cssmatcher: Rewrite nth-child matcher

Instead of trying to be smart, be stupid but correct.

Fixes nth-child reftest.

cssmatcher: Simplify code

Instead of copy/paste, pass the only difference as an argument

The code still doesn't work right, but at least it's easier to fix now.

gtk: Add deprecation guards for newest pango deprecations

testsuite: Add deprecation guards

Updated Spanish translation

3.17.3

Updated Spanish translation

3.17.2

Dist the gtk-builder-tool test script

examples: Clean up the standalone Makefiles

Do not hardcode GCC as the compiler; use $(shell) expansion instead of
backticks; split the built source into its own variable.

entry: Set up text drag icon within drag_begin()

the drag_begin() vmethod is meant for this, and the internal DnD code
will set up a drag icon if ::drag_begin didn't do so, which means
we are first getting a "default" icon, and then replacing it with the
text surface.

This is completely harmless in X11, but causes issues on wayland as
the DnD icon window is expected to remain unchanged during DnD there.

https://bugzilla.gnome.org/show_bug.cgi?id=748763

notebook: Fix assert in ::drag-motion

The drag destination might be empty, we shouldn't be checking whether
it contains pages at all. Instead, check the source notebook, which
ought to have a selected page if you're dragging something from there.

https://bugzilla.gnome.org/show_bug.cgi?id=749893

Updated German translation

Use built-in gtk-update-icon-cache

To generate the icon cache files.

We want to avoid a dependency loop if possible; additionally, on some
Debian-based systems gtk-update-icon-cache maps to the GTK2 version of
the utility and the GTK3 version is renamed to
gtk-update-icon-cache-3.0.

To avoid a build dependency on GTK2, use the binary that we just built
in-tree.

https://bugzilla.gnome.org/show_bug.cgi?id=749593

parse-sass: don't hardcode bash path

It's /bin/bash on some systems; just use /bin/sh instead.

https://bugzilla.gnome.org/show_bug.cgi?id=749593

tests: wait for draw before fetching tree view style

On some slower machines (e.g. an ARM OBS builder), this test is failing
with a race condition where we're trying to fetch the style before it's
applied.

https://bugzilla.gnome.org/show_bug.cgi?id=749593

scrolledwindow: fix copy/paste typo

fc2830394895f236352bea2024dda0e962dd2c53 refactored code into the
get_scroll_unit() function but introduced a copy/paste typo.

placessidebar: escape tooltip text before setting it

gtk_tree_view_set_tooltip_column() specifies that markup in the text
should be escaped.
This fixes critical warnings when hovering over items in the sidebar for
bookmarks that have markup characters in their names.

https://bugzilla.gnome.org/show_bug.cgi?id=719683

scrolledwindow: Trigger builtin kinetic deceleration on libinput devices

The libinput driver will send a 0/0 scroll event on touchpads and other
devices where it knows scrolling stopped for sure. Use these events to
trigger kinetic scrolling from there.

The mechanism is similar to GtkGestureSwipe, we keep a backlog of the
latest dx/dy till a previous point in time, and calculate the final
velocities from there, with the difference we're dealing with scroll
units, and not pixel distances.

https://bugzilla.gnome.org/show_bug.cgi?id=749770

scrolledwindow: Refactor scroll unit guessing into a separate function

Makes it clearer, and will be used in further places.

https://bugzilla.gnome.org/show_bug.cgi?id=749770

Fixes to Catalan translation

scrolledwindow: reset more Indicator state on ::unmap

If a GtkScrolledWindow is just unmapped and promptly mapped again, the
indicators are left in a semi-visible state, so the GdkWindow isn't raised
properly above scrolledwindow content. This inconsistent state went away
the next time the indicator is hidden.

So, reset all state about indicator window visibility, animation
progress and conceil timer on ::unmap, this will be enough to make the
indicators start out hidden like on newly created scrolledwindows.

gdk: Fix a typo

Fix primary monitor determination with XRANDR 1.5

This was an oversight in the previous patch.

gtk3: add randr 1.5 monitor support

This patch introduces support for using the newly introduced
monitor objects in the XRandR protocol. These objects are meant
to be used to denote a set of rectangles representing a logical
monitor, and are used to hide details like monitor tiling and
virtual gpu outputs.

This uses the new objects instead of crtc/outputs objects when
they are available to create the monitor lists. X server 1.18
is required on the server side for randr 1.5.

https://bugzilla.gnome.org/show_bug.cgi?id=749561

iconview: Avoid usage of gdk_window_get_device_position() during DnD

Just use the last coordinates given on XdndPosition/drag_motion() in
order to trigger scrolling.

When running on Xwayland, the pointer position is unknown at this
stage on the X11 side, so the coordinates given here are bogus.
This change avoids both roundtrips and this situation.

https://bugzilla.gnome.org/show_bug.cgi?id=749679

treeview: Avoid usage of gdk_window_get_device_position() during DnD

Just use the last coordinates given on XdndPosition/drag_motion() in
order to trigger scrolling.

When running on Xwayland, the pointer position is unknown at this
stage on the X11 side, so the coordinates given here are bogus.
This change avoids both roundtrips and this situation.

https://bugzilla.gnome.org/show_bug.cgi?id=749679

textview: Avoid usage of gdk_window_get_device_position() during DnD

Just use the last coordinates given on XdndPosition/drag_motion() in
order to trigger scrolling.

When running on Xwayland, the pointer position is unknown at this
stage on the X11 side, so the coordinates given here are bogus.
This change avoids both roundtrips and this situation.

https://bugzilla.gnome.org/show_bug.cgi?id=749679

gtk3-demo: Fix Visual Studio Builds

Add the new cursors demo to the projects, so that gtk-demo will properly
build.

GDK: Add cursor theme support to W32 backend

Load themed cursors from the same places they are loaded on freedesktop systems,
but use W32 API functions to do so (works for .cur/.ani cursors instead of X
cursors).

Refactor the code for cursor handling. Prefer loading cursors by name.

Do not load actual cursors when loading the theme. Find the files and remember
the arguments/calls for loading them instead. Keeping HCURSOR instance in the
hashmap would result in multiple GdkCursors using the same HCURSOR. Given that
we use DestroyCursor() to off them, this would cause problems (at the very
least - DestroyCursor() would fail).

Store GdkCursor instances in a cache. Update cached cursors when theme changes.

Recognize "system" theme as a special (and default) case. When it is set,
prefer system cursors and fall back to Adwaita cursors and (as a last resort)
built-in X cursors. Otherwise prefer theme cursors and fall back to system and
X cursors.

Force GTK to use "left_ptr" cursor when no cursor is set. Using NULL makes
it use the system default "arrow", which is not the intended behaviour when
a non-system theme is selected.

Ignore cursor size setting and query the OS for the required cursor size, as
Windows (almost) does not allow setting cursors of arbitrary size.

https://bugzilla.gnome.org/show_bug.cgi?id=749287

GDK: Use GdkCursor objects to keep track of W32 cursors, not HCURSOR

In particular this means that cursors are disposed of by the way of
g_object_unref(), not DestroyCursor (which is documented to not to be
used on certain kinds of cursors, and we can't tell which is which).

It should also alleviate any concerns about destroying cursors that
are still in use by other windows, except for cases where we would
somehow get our hands on a HCURSOR that someone else is using and we
make a GdkCursor out of it and later unref and finalize it while it
is still in use.

It also removes the need to call CopyCursor(), which makes animated
cursors into non-animated ones as a side-effect (supposed to be a bug,
but try explaining that to MS). Now cursors should be animated (if
the are set up as such in the OS).

https://bugzilla.gnome.org/show_bug.cgi?id=697477